test(harness): establish shared live E2E kernel for Codex and Claude - #2159
Merged
Conversation
Loading
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
Issue #1717 の段階展開のうち、共通 live E2E kernel と Codex / Claude Code の各 transport を実装します。
背景
既存の
codex-exec-live.tsにはlive検証の安全境界が集約されていましたが、他transportではgate、認証隔離、cleanup、skip reasonが分散していました。本変更ではtransport自体を統一せず、policy/lifecycleだけを共有kernelへ切り出します。これにより、明示opt-inがない実行やGitHub Actions上の実行をprocess起動前に拒否し、source home・認証設定・ユーザーhooksをchildへ伝播しない契約を各adapterで同じ形に固定できます。
影響
検証
bun run buildbun run typecheckbun run source-only:checkbun run lint（既存のcomplexity警告のみ）残作業
このDraft PRはIssue #1717全体を閉じません。Kimi Code、Kiro CLI/IDE、Cursor、OpenCodeについては、共通policy/lifecycleへの接続または実測に基づく後続Issueへの分離が必要です。
Related to #1717
Note
Low Risk
Changes are documentation and governance metadata under
spaces/default/with no application runtime or auth paths touched; main review cost is volume and consistency of election/re-scan claims (e.g. commit counts, CI evidence), not production breakage.Overview
This diff persists the audit trail for the
260803-harness-live-e2eintent (Issue #1717 live E2E policy/adapters), not harness TypeScript in the slice shown here.It introduces
amadeus-state.mdmarking the workflow completed atbuild-and-test(14 stages done, Construction verified, mirror receipts through workflow-completed).It adds
codekb/.../re-scans/260803-harness-live-e2e.md, grounding design on observed commit52a082af7(46 commits from base): per-surface live-gate/credential gaps, recommended common policy/lifecycle vs transport-adapter split, and phased rollout notes aligned with the PR’s kernel direction.It registers 14 new
E-HLE-*elections underspaces/default/elections/(intent-capture through build-and-test, including units-generation reruns and a follow-upE-HLE-RA13-ZEROafterE-HLE-RA13split). Each package ships ballots, ledgers, tallies, timelines, views, andrecord.md; almost all outcomes are “0 new project/team rules or sensors”—learnings stay on the intent record. Notable process detail:E-HLE-RES13went hold → reopen with amended ballots after git ancestry disputes;E-HLE-BT13/E-HLE-FD13ballots carry reservations (CI re-run evidence, ADR-009 wording).elections.jsonlists all new election dirs asrecorded.Reviewed by Cursor Bugbot for commit cd02a55. Configure here.
Summary by CodeRabbit
新機能
ドキュメント
テスト